- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.2k
eip7928: introduce specs for block access lists #4526
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
eip7928: introduce specs for block access lists #4526
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this PR should include tests
| @nerolation it's pretty clear that this PR was done by an LLM. Mind cleaning up after it? Eg this makes no sense and the linter check is still failing: 
 | 
        
          
                tests/core/pyspec/eth2spec/test/eip7928/test_block_access_list.py
              
                Outdated
          
            Show resolved
            Hide resolved
        
      | Thanks @jtraglia! | 
Co-authored-by: Justin Traglia <[email protected]>
Add EIP-7928: Block-Level Access Lists
This PR implements EIP-7928, which introduces block-level access lists. The implementation extends the
ExecutionPayloadandExecutionPayloadHeaderstructures to include a newblock_access_listfield that contains RLP-encoded access list data. This allows execution clients to provide and validate block-level access lists through the existing Engine API interface without requiring new methods.make lintto check formatting (completed with some unrelated prysm warnings)make testto check tests (running)Related to Ethereum EIP-7928: https://eips.ethereum.org/EIPS/eip-7928
Summary of Changes
ExecutionPayloadcontainer withblock_access_list: BlockAccessListfieldExecutionPayloadHeadercontainer withblock_access_list_root: Rootfieldprocess_execution_payloadto compute and store the access list rootfork.mdfor upgrading to EIP-7928